docs: add floci integration documentation - #1373
Merged
Merged
Conversation
- Add Floci UI web console integration examples for C# and TypeScript - Include customization options for container name and host port - Add note about Floci's built-in UI mechanism vs Aspire resource - Enhance Quarkus configuration section with standard location reference - Add clarification about FLOCI_ prefixed environment variables - Improve TLS/HTTPS section structure with better documentation - Detail port routing behavior for TLS connections
edmondshtogu (edmondshtogu)
force-pushed
the
floci
branch
from
August 5, 2026 14:39
1319131 to
9b623a6
Compare
Alistair Matthews (alistairmatthews)
requested changes
Aug 5, 2026
Alistair Matthews (alistairmatthews)
left a comment
Alistair Matthews (alistairmatthews)
left a comment
Collaborator
There was a problem hiding this comment.
This is great. Just a few minor style and grammar suggestions. Thanks!
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
Co-authored-by: Alistair Matthews <alistairwebdojo@live.com>
edmondshtogu (edmondshtogu)
requested a review
from Alistair Matthews (alistairmatthews)
August 7, 2026 12:42
David Pine (IEvangelist)
marked this pull request as ready for review
August 18, 2026 02:29
David Pine (IEvangelist)
requested review from
Alistair Matthews (alistairmatthews)
and
a lite review from Copilot
and removed request for
Alistair Matthews (alistairmatthews)
August 18, 2026 02:29
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new Aspire integration documentation page for Floci and wires it into the frontend’s integration navigation and package metadata lists so it shows up in the site and integration lookups.
Changes:
- Added new Floci integration documentation page under Compute integrations.
- Registered the Floci integration in the site’s integration docs mapping and integrations metadata list.
- Added Floci to the Compute & hosting sidebar topic list.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/frontend/src/data/integration-docs.json | Maps the Floci integration package ID to the new docs route. |
| src/frontend/src/data/aspire-integrations.json | Adds Floci to the integrations catalog metadata (title/description/tags/version). |
| src/frontend/src/content/docs/integrations/compute/floci.mdx | New integration documentation page content for Floci. |
| src/frontend/src/assets/icons/floci-icon.svg | Adds Floci icon asset. |
| src/frontend/src/assets/icons/floci-icon-light.svg | Adds Floci light icon asset used by the docs page. |
| src/frontend/config/sidebar/integrations.topics.ts | Adds Floci to the Compute sidebar navigation. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Adam Ratzman (adamint)
left a comment
Adam Ratzman (adamint)
left a comment
Member
There was a problem hiding this comment.
I do not have anything to add beyond the existing review threads.
Alistair Matthews (alistairmatthews)
left a comment
Alistair Matthews (alistairmatthews)
left a comment
Collaborator
There was a problem hiding this comment.
Thanks for addressing my previous review.
Please address my one suggestion and Copilot's comment about unclosed tab and tabitem blocks. Then this is ready to go.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Alistair Matthews <a.matthews@reddoorcollaborative.com>
Contributor
Author
|
Thanks for reviewing my PR. I fixed the issues with tabs, and now I am waiting for the package to be published so it exist and as soon as is present we can merge it. |
Alistair Matthews (alistairmatthews)
approved these changes
Aug 20, 2026
Alistair Matthews (alistairmatthews)
left a comment
Alistair Matthews (alistairmatthews)
left a comment
Collaborator
There was a problem hiding this comment.
Ready to merge.
David Pine (IEvangelist)
approved these changes
Aug 20, 2026
David Pine (IEvangelist)
enabled auto-merge (squash)
August 20, 2026 16:08
David Pine (IEvangelist)
added a commit
that referenced
this pull request
Aug 20, 2026
CommunityToolkit.Aspire.Hosting.Floci is not yet published on nuget.org, so the integration data refresh correctly drops it from aspire-integrations.json. The docs added in #1373 (doc mapping, page, sidebar entry, and icons) still referenced a package absent from the catalog, which failed the update-integrations guardrail test ("only maps packages in the current integration catalog"). Remove the Floci docs footprint until the package ships. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue: floci-io/floci#1242
Integration: CommunityToolkit/Aspire#1448